Skip to content

Conversation

@abrookins
Copy link
Collaborator

@abrookins abrookins commented Dec 9, 2025

Fixes #97

Copilot AI review requested due to automatic review settings December 9, 2025 17:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds automated publishing infrastructure for the agent-memory-server package to PyPI. It introduces a GitHub Actions workflow that automatically publishes the server to PyPI when version tags are pushed, mirroring the existing pattern established for the agent-memory-client package. The PR also bumps the server version to 0.12.4 in preparation for release.

Key changes:

  • New GitHub Actions workflow for automated PyPI publishing with TestPyPI support
  • Python script for safe tag creation and push operations with dry-run mode
  • Version bump to 0.12.4 for the agent-memory-server package

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/agent-memory-server.yml New CI/CD workflow that tests, builds, and publishes the server package to PyPI or TestPyPI based on git tags
scripts/tag_and_push_server.py Helper script to create and push release tags with safety features like dry-run mode and force flag
agent_memory_server/__init__.py Version bump from 0.12.3 to 0.12.4

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


args = parser.parse_args()

# Change to project root directory
Copy link

Copilot AI Dec 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The import os statement should be moved to the top of the file with other imports (around line 14-18). Importing modules inside functions or conditional blocks is generally discouraged as it can make code harder to understand and debug. Move it to the module-level imports section.

Copilot uses AI. Check for mistakes.
@abrookins abrookins merged commit 4cd4137 into main Dec 9, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support running MCP server via uvx

2 participants